Skip to content

backport to 1.14: Prevent SEGFAULT when disabling listener (#13515)#13945

Merged
lizan merged 11 commits intoenvoyproxy:release/v1.14from
cpakulski:rel-14.6
Nov 11, 2020
Merged

backport to 1.14: Prevent SEGFAULT when disabling listener (#13515)#13945
lizan merged 11 commits intoenvoyproxy:release/v1.14from
cpakulski:rel-14.6

Conversation

@cpakulski
Copy link
Contributor

Commit Message:
This prevents the stop_listening overload action from causing
segmentation faults that can occur if the action is enabled after the
listener has already shut down.

Risk Level: low
Testing: unit tests
Docs Changes: none
Release Notes: Yes
Fixes #13514

cpakulski and others added 11 commits October 21, 2020 23:09
Signed-off-by: Christoph Pakulski <christoph@tetrate.io>
Signed-off-by: Christoph Pakulski <christoph@tetrate.io>
Signed-off-by: Christoph Pakulski <christoph@tetrate.io>
…nto rel-14.6

Signed-off-by: Christoph Pakulski <christoph@tetrate.io>
This prevents the stop_listening overload action from causing
segmentation faults that can occur if the action is enabled after the
listener has already shut down.

Signed-off-by: Alex Konradi <akonradi@google.com>
Signed-off-by: Christoph Pakulski <christoph@tetrate.io>
This prevents the stop_listening overload action from causing
segmentation faults that can occur if the action is enabled after the
listener has already shut down.

Signed-off-by: Alex Konradi <akonradi@google.com>
Signed-off-by: Christoph Pakulski <christoph@tetrate.io>
Signed-off-by: Christoph Pakulski <christoph@tetrate.io>
…l-14.6

Signed-off-by: Christoph Pakulski <christoph@tetrate.io>
Signed-off-by: Christoph Pakulski <christoph@tetrate.io>
Signed-off-by: Christoph Pakulski <christoph@tetrate.io>
Signed-off-by: Christoph Pakulski <christoph@tetrate.io>
@antoniovicente antoniovicente self-assigned this Nov 9, 2020
@antoniovicente antoniovicente self-requested a review November 9, 2020 19:29
@antoniovicente
Copy link
Contributor

Change looks correct. Waiting for CI before approving and merging.

@cpakulski cpakulski marked this pull request as ready for review November 9, 2020 21:26
@lizan lizan merged commit 1160bd6 into envoyproxy:release/v1.14 Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants